Skip to content

fix(ci): use live PR title in feat-minor-bump-gate (#3240)#3242

Merged
OneStepAt4time merged 1 commit into
developfrom
fix/3240-gate-frozen-title
May 12, 2026
Merged

fix(ci): use live PR title in feat-minor-bump-gate (#3240)#3242
OneStepAt4time merged 1 commit into
developfrom
fix/3240-gate-frozen-title

Conversation

@OneStepAt4time

Copy link
Copy Markdown
Owner

Summary

Fixes #3240feat-minor-bump-gate reads frozen webhook title instead of live PR title.

Problem

The feat-minor-bump-gate CI job uses pr.title from the frozen webhook payload for the isFeat regex check, while already fetching livePr from the API for labels. Title renames via GitHub UI or after amend+force-push don't take effect.

Fix

Moved the livePr API fetch before the title check and changed pr.titlelivePr.title. Single API call now provides both live title and live labels.

Impact

Verification

# Aegis session: 4033fa18-4513-4ce6-81e7-7c50558fe5a1
# Commit: 74cc00f7
# Branch: fix/3240-gate-frozen-title → develop
# Change: 1 file, 9 insertions(+), 8 deletions(-)

CI workflow changes don't have unit tests — verified by reading the diff. The logic is identical to before except pr.titlelivePr.title and the API call is moved up.

@aegis-gh-agent aegis-gh-agent Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved -- all CI green.

Surgical fix: moved livePr API fetch before title check, pr.title -> livePr.title. Single API call provides both live title and live labels.

This fixes the exact bug that blocked #3222 and #3238. Title renames will now take effect on re-run without force-push.

Bonus: developed through Aegis (dogfooding!).

@OneStepAt4time OneStepAt4time merged commit 3126220 into develop May 12, 2026
18 checks passed
@OneStepAt4time OneStepAt4time deleted the fix/3240-gate-frozen-title branch May 12, 2026 11:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant